Added a new cmake_build for generating a debian#855
Added a new cmake_build for generating a debian#855tarunkartik wants to merge 3 commits intoBehaviorTree:masterfrom
Conversation
|
@facontidavide is there something blocking this merge or is this a feature that you dont foresee people using? |
|
Is this change tested on Windows? We don't want to break it there |
|
I have not tested it on windows. I can do that. It looks like the repo already has a pipeline to test against windows. Are there any instructions on how to trigger them? |
| option(BTCPP_UNIT_TESTS "Build the unit tests" ON) | ||
| option(BTCPP_GROOT_INTERFACE "Add Groot2 connection. Requires ZeroMQ" ON) | ||
| option(BTCPP_SQLITE_LOGGING "Add SQLite logging." ON) | ||
| option(CMAKE_BUILD "Build using CMAKE" OFF) |
There was a problem hiding this comment.
what is the purpose of adding this? Shouldn't it be the default ON?
There was a problem hiding this comment.
The intent is that the users would turn this ON only when they need standalone debians that do not depend on ROS (catkin/ament). That was my requirement for a project I have been working on where I am using BTCPP with a custom RTOS.
Maybe the variable could be named 'GENERATE_DEBS' instead of 'CMAKE_BUILD', if that makes it clearer?
There was a problem hiding this comment.
Do you suggest renaming the variable?
|
not interested in supporting this directly |
If you want to generate a standalone behaviortree_cpp debian that you can install and link your project against,
Replace {Number_of_cores} with how many cores you would like to use to build.
.debfile inside the build folder.apt install /path/to/.deb